@nickmccurdy @MirrorBytes @mihar-22 do you think we could merge this? Thanks!

LGTM! Need to pull branch and test but I haven't been up to date with svelte-testing-library in a while so probably best to defer to @nickmccurdy.

Hi, sorry for the delay. The svelte-jester update sounds good to me though I'm not experienced with Svelte.

I ran the tests locally and they failed with the same issues that I fixed in #161 (Jest 27 no longer loads JSDOM by default). I added those fixes to this branch and the tests are now passing locally. I also cleaned up the test code, I don't think DOM cleanup is necessary since Jest creates new environments between test runs anyway.

Please review when you have a chance.

I got CI working here, but unfortunately it seems like svelte-jester 2 requires Node 14, and Node 12 is still officially supported until April. This package still supports all the way back to Node 8 though. How should we handle this?

Ok, I sent a PR to fix that: svelteness/svelte-jester#83

@nickmccurdy I've updated this PR to svelte-jester 2.1.4, which supports older versions of Node

Thanks for the fix! Looks great, we seem to have cut test times in half.

@all-contributors add @benmccann for tests

@nickmccurdy

I've put up a pull request to add @benmccann! 🎉

@all-contributors add @mihar-22 for review

@nickmccurdy

I've put up a pull request to add @mihar-22! 🎉

🎉 This PR is included in version 3.1.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀